Skip to content

Sylius 2.x compatibility #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 28 commits into from
Nov 13, 2025
Merged

Sylius 2.x compatibility #4

merged 28 commits into from
Nov 13, 2025
+315 −260

Conversation

@bourtitom
Copy link
Contributor

@bourtitom bourtitom commented Jun 27, 2025 •
edited by maximehuran
Loading

Add Sylius 2.x compatibility

  • README compatibilities updated
  • README link to branch for Sylius 1.x compatibility
  • README update images
  • UPGRADE small guide
  • Update recipe in Mbiz repo
  • Settings works
  • Login page is customized
  • Logo in sidebar is displayed
  • Custom CSS works
  • modify the Sylius version and its dependencies
  • modify the structure file to the project root
  • modify the alias and path to the project root
  • Check dist files
  • Create 1.x and 2.x branch, switch PR base, and change default plugin branch

demo :
image

image image image

@bourtitom bourtitom marked this pull request as draft June 27, 2025 15:07
@bourtitom bourtitom marked this pull request as ready for review July 8, 2025 09:35
@madamebiz madamebiz added the Status: needs review Needs review label Jul 8, 2025
delyriand
delyriand suggested changes Jul 8, 2025
View reviewed changes
@madamebiz madamebiz added Status: needs update This Issue/PR needs update and removed Status: needs review Needs review labels Jul 8, 2025
@bourtitom bourtitom requested a review from delyriand July 10, 2025 14:56
@madamebiz madamebiz added the Status: needs review Needs review label Jul 10, 2025
@madamebiz madamebiz removed the Status: needs update This Issue/PR needs update label Jul 10, 2025
@lanfisis lanfisis removed their request for review August 1, 2025 15:41
@maximehuran maximehuran changed the title Feat/upgrade 2.0 Sylius 2.x compatibility Aug 18, 2025
delyriand
delyriand suggested changes Sep 1, 2025
View reviewed changes
templates/security/common/login.html.twig Outdated
Copy link

@delyriand delyriand Sep 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❔ question: Why are there so many elements in this template? I think you can use ‘sylius_admin.security.login.page.content’ to just add the JS for the background.‏

@madamebiz madamebiz added Status: needs update This Issue/PR needs update and removed Status: needs review Needs review labels Sep 1, 2025
@maximehuran maximehuran removed their request for review September 26, 2025 14:34
@maximehuran maximehuran changed the base branch from master to 2.x November 12, 2025 15:44
@maximehuran maximehuran self-requested a review November 12, 2025 16:43
@madamebiz madamebiz added Status: needs review Needs review and removed Status: needs update This Issue/PR needs update labels Nov 12, 2025
@maximehuran maximehuran merged commit df47357 into monsieurbiz:2.x Nov 13, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maximehuran maximehuran maximehuran approved these changes

+1 more reviewer

@delyriand delyriand delyriand requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Status: needs review Needs review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bourtitom @delyriand @maximehuran @madamebiz